Skip to content

Conversation

trueberryless
Copy link
Contributor

@trueberryless trueberryless commented Sep 24, 2025

Description (required)

Adds Cloudflare to sponsor list in Sidebar.

  • The svg got optimized with SVGOMG
  • The text of the svg will adapt to the theme (dark/light) thanks to currentColor
  • The width of the svg got set to 130 as a start value and I am open for suggestions

Please let me know if the position / size of the SVG should be adapted anyhow.

Related issues & labels (optional)

Copy link

netlify bot commented Sep 24, 2025

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 1a417c1
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68dbacaf839d8500082272b5
😎 Deploy Preview https://deploy-preview-12410--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

{/* Spacer element to push following logos onto a new row. */}
<div style="flex-basis: 100%"></div>

<a href="https://www.cloudflare.com/?utm_source=astro&utm_medium=partner" aria-label="Cloudflare">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not know the correct UTM url.
This needs to be updated to the correct URL by some core maintainer I think. Either leave a suggestion or commit the corrected URL 🙏

@trueberryless trueberryless changed the title feat: add cloudflare sponsor Adds Cloudflare to the list of sponsors Sep 24, 2025
@sarah11918
Copy link
Member

sarah11918 commented Sep 26, 2025

Just a heads up that Chris is the one who does this for our official partners, and has his own process for this. (And yes, we have additional information that we use that would mean a community member would be unable to properly complete this and why the core team handles updates like this.)

So, when he's back, he may end up closing this and making his own PR, or he may work with this one! But that's why nothing has happened here yet! 🙌

Copy link
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting this together @trueberryless! I tweaked it slightly to move Cloudflare higher in the list and ensure the mobile menu layout looks good. Should be good to go now 🚀

@delucis delucis added the Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! label Sep 30, 2025
@delucis delucis merged commit bce8a4d into withastro:main Sep 30, 2025
10 checks passed
@trueberryless trueberryless deleted the tbl-cloudflare-sponsor branch September 30, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants